Skip to content

Add RHEL support and include Linux distro in cache keys#1323

Merged
HarithaVattikuti merged 5 commits into
actions:mainfrom
priyagupta108:add-rhel-support
Jun 18, 2026
Merged

Add RHEL support and include Linux distro in cache keys#1323
HarithaVattikuti merged 5 commits into
actions:mainfrom
priyagupta108:add-rhel-support

Conversation

@priyagupta108

Copy link
Copy Markdown
Contributor

Description:

  • Adds RHEL support for Python version resolution and installation.
  • Includes the Linux distro (version + name) in the cache keys and aligns them across pip, pipenv, and poetry.

Impact:
One-time cache regeneration for Linux pipenv/poetry users on the first run. Windows and macOS keys are unchanged.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@priyagupta108 priyagupta108 requested a review from a team as a code owner June 17, 2026 06:14
Copilot AI review requested due to automatic review settings June 17, 2026 06:14
@priyagupta108 priyagupta108 self-assigned this Jun 17, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds RHEL-aware Python release resolution and standardizes Linux cache keys by incorporating Linux distro name + version across pip/pipenv/poetry (while keeping Windows/macOS keys unchanged).

Changes:

  • Add RHEL-specific manifest filtering during Python version resolution.
  • Add a shared getLinuxInfoKeySegment() helper and use it to include Linux distro info in pip/pipenv/poetry cache keys.
  • Make Linux distro detection more resilient by falling back from lsb_release to /etc/os-release.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/utils.ts Adds fallback Linux distro detection via /etc/os-release when lsb_release fails.
src/install-python.ts Adds RHEL detection and custom manifest matching for platform: rhel.
src/cache-distributions/cache-distributor.ts Introduces shared Linux distro cache-key segment helper (incl. RHEL major-version logic).
src/cache-distributions/pip-cache.ts Switches pip cache keys to use the shared Linux distro key segment helper.
src/cache-distributions/pipenv-cache.ts Adds Linux distro segment to pipenv cache keys via shared helper.
src/cache-distributions/poetry-cache.ts Adds Linux distro segment to poetry cache keys via shared helper.
dist/setup/index.js Regenerated bundled output reflecting the source changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/utils.ts
Comment thread src/cache-distributions/cache-distributor.ts Outdated
Comment thread src/install-python.ts
Comment thread src/cache-distributions/pipenv-cache.ts
priya-kinthali
priya-kinthali previously approved these changes Jun 17, 2026
@HarithaVattikuti HarithaVattikuti merged commit 0cb1a84 into actions:main Jun 18, 2026
1401 of 1410 checks passed
ajgon pushed a commit to deedee-ops/schemas that referenced this pull request Jun 24, 2026
…0) (#5)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/setup-python](https://github.com/actions/setup-python) | action | minor | `v6.2.0` → `v6.3.0` |

---

### Release Notes

<details>
<summary>actions/setup-python (actions/setup-python)</summary>

### [`v6.3.0`](https://github.com/actions/setup-python/releases/tag/v6.3.0)

[Compare Source](actions/setup-python@v6.2.0...v6.3.0)

#### What's Changed

##### Enhancement

- Add RHEL support and include Linux distro in cache keys by [@&#8203;priyagupta108](https://github.com/priyagupta108) in [#&#8203;1323](actions/setup-python#1323)
- Fix pip cache error handling on Windows by [@&#8203;priyagupta108](https://github.com/priyagupta108) in [#&#8203;1040](actions/setup-python#1040)

##### Dependency update

- Upgrade minimatch from 3.1.2 to 3.1.5 by [@&#8203;dependabot](https://github.com/dependabot) in [#&#8203;1281](actions/setup-python#1281)
- Upgrade actions dependencies by [@&#8203;gowridurgad](https://github.com/gowridurgad) with [@&#8203;Copilot](https://github.com/Copilot) in [#&#8203;1303](actions/setup-python#1303)
- Upgrade [@&#8203;actions/cache](https://github.com/actions/cache) to 5.1.0, log cache write denied by [@&#8203;jasongin](https://github.com/jasongin) in [#&#8203;1324](actions/setup-python#1324)
- Upgrade dependency versions and test workflow configuration by [@&#8203;HarithaVattikuti](https://github.com/HarithaVattikuti) in [#&#8203;1322](actions/setup-python#1322)

##### Documentation

- Update advanced-usage.md by [@&#8203;Dunky-Z](https://github.com/Dunky-Z) in [#&#8203;811](actions/setup-python#811)

#### New Contributors

- [@&#8203;gowridurgad](https://github.com/gowridurgad) with [@&#8203;Copilot](https://github.com/Copilot) made their first contribution in [#&#8203;1303](actions/setup-python#1303)
- [@&#8203;jasongin](https://github.com/jasongin) made their first contribution in [#&#8203;1324](actions/setup-python#1324)
- [@&#8203;Dunky-Z](https://github.com/Dunky-Z) made their first contribution in [#&#8203;811](actions/setup-python#811)

**Full Changelog**: <actions/setup-python@v6...v6.3.0>

</details>

---

### Configuration

📅 **Schedule**: (in timezone Europe/Warsaw)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4yNDAuMCIsInVwZGF0ZWRJblZlciI6IjQzLjI0MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIiwibGFiZWxzIjpbXX0=-->

Reviewed-on: https://git.ajgon.casa/deedee/schemas/pulls/5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants